gtk: use the new modifier API instead of GTK_DEFAULT_ACCEL_MOD_MASK

in gtkimcontextsimple and gtktreeview, which are the easy places,
but also true bugfixes, because on OSX we can't use the virtual META
here, we needs MOD2 as delivered in key events.

Updated Galician translations

GtkComboBox: Add the format-entry-text signal.

This provides more control over how the selected item is shown in
the entry.
Bug #631167

GtkTreeViewColumn: dispose: Remove from the treeview

This handles the case (with gtkmm) that the GtkTreeViewColumn is
destroyed before the GtkTreeView is destroyed.
Bug #659672

Bug 628936: Minimal change to pass introspection.

gobject_introspection's G-ir-scanner doesn't like the -xobjective-c
option needed to compile ObjectiveC features in quartz implementations
of certain gtk functions. This rearranges the compiler flag environment
variables in Makefile so that G-ir-scanner doesn't see them.

Fixed gtk_cell_area_stop_editing() to properly notify when editing is not canceled

This fixes bug 653289, in gtk+-2 gtk_tree_view_stop_editing() was previously
responsible for notifying with gtk_cell_editable_editing_done(), this detail
was missed when porting the cell renderer management to GtkCellArea.

reftests: add a reftest for the CSS :only-child selector

https://bugzilla.gnome.org/show_bug.cgi?id=660833

all: add an 'only-child' pseudo-class selector

Add an :only-child pseudo-class selector, which selects single children
of a parent container.

https://bugzilla.gnome.org/show_bug.cgi?id=660833

appchooser: make the "Find applications online" follow search state

Set the button as insensitive while searching, so that it can't be
accidentally clicked twice.

https://bugzilla.gnome.org/show_bug.cgi?id=651818

appchooser: add a GCancellable to the search operation

So that we can cancel it when the dialog is destroyed.

https://bugzilla.gnome.org/show_bug.cgi?id=651818

Updated Spanish translation

Fix commit 61d6c73f186a08cd55852c0a44a1704a6fbbbc39

Where the commit suggested "somebody should review it [the test]",
it would have been useful to test this particular commit instead...

Bug 659022 - gtk_tree_model_filter_clear_cache_helper: assertion failed

This bug is resolved by fixing two things in
gtk_tree_model_filter_row_deleted():

  (1) It is possible for an elt to have elt->visible_siter == NULL, when
  it is deleted.  Only call g_sequence_remove() if this pointer is
  non-NULL.

  (2) For the case len (level->seq) > 1, free the elt->children level
  if non-NULL.  Failing to do this means the level will stick around.
  If this child level was not referenced, it will still have a zero
  ref count on its parent which cannot be removed!

For both bugs unit tests have been added in the preceding commit.

Add two more unit tests for bug 659022

Both pointing out problems in gtk_tree_model_filter_row_deleted().

Fix first part of bug 659022

Push creation of path into if clause, the path cannot be created anyway
(and would be meaningless otherwise) if the parent is not visible.

Add unit test for part 1 of bug 659022

This tests a case where gtk_tree_model_filter_row_changed() is run
for a node with visible_siter != NULL, however its parent has
visible_iter == NULL. For this case, no valid paths can be generated
as the node cannot be accessed (because the parent is invisible).

box: add some missing includes

iconcache: Fix gcc warning

filesystemmodel: Add a missing check

Paths with depth > 1 should return FALSE instead of pretending the depth
is 1.

Bug 660730: Use GStatBuf for portability

Thanks to Kean Johnston for pointing this out.

There are a few places in GTK that use "struct stat",
and then g_stat(), rather than using GStatBuf.This breaks things on
Windows. Since the size of struct stat can vary depending on other
flags specified, this has the potential to cause overwrites and is
trivial to fix.

Based on patch submitted by Kean Johnston

gdk: Plug leak in gdk_display_pointer_is_grabbed

gdk_x11_device_manager_core_list_devices returns a new allocated
list, which has to be freed.

valgrind output:

==18686== 160,176 (80,088 direct, 80,088 indirect) bytes in 3,337 blocks are definitely lost in loss record 25,347 of 25,378
==18686==    at 0x4C256DD: malloc (in /usr/lib64/valgrind/vgpreload_memcheck-amd64-linux.so)
==18686==    by 0x6CD7752: g_malloc (in /lib64/libglib-2.0.so.0.3000.0)
==18686==    by 0x6CEE2B6: g_slice_alloc (in /lib64/libglib-2.0.so.0.3000.0)
==18686==    by 0x6CCB37D: g_list_prepend (in /lib64/libglib-2.0.so.0.3000.0)
==18686==    by 0x654CADA: gdk_x11_device_manager_core_list_devices (gdkdevicemanager-core-x11.c:836)
==18686==    by 0x6531489: gdk_display_pointer_is_grabbed (gdkdisplay.c:1270)
==18686==    by 0x5162E1E: filter_func (ui.c:140)
==18686==    by 0x6558B50: gdk_event_apply_filters (gdkeventsource.c:83)
==18686==    by 0x6558CB3: _gdk_x11_display_queue_events (gdkeventsource.c:197)
==18686==    by 0x6530680: gdk_display_get_event (gdkdisplay.c:311)
==18686==    by 0x65589F1: gdk_event_source_dispatch (gdkeventsource.c:356)
==18686==    by 0x6CD0A0E: g_main_context_dispatch (in /lib64/libglib-2.0.so.0.3000.0)

https://bugzilla.gnome.org/show_bug.cgi?id=660676

Updated Esperanto translation

__SIZEOF_INT__ and __SIZEOF_POINTER__ may not be defined.

Protect __SIZEOF_INT__ and __SIZEOF_POINTER__ with an ifdef to prevent
warnings about them not being defined when including gtktextattributes.h.

https://bugzilla.gnome.org/show_bug.cgi?id=660619

gtkbox: Move private functions to private header

Add 'Since 3.4' indices to the docs

Add new api to gdk.symbols

This keeps 'make check' happy.
And if 'make check' is happy, so am I.

gdkcreen-x11: Use GdkRGBA instead GdkColor

Updated Vietnamese translation

po/vi: import from Damned Lies

uimanager: Deprecate add-tearoffs property

Don't use it. Please.

menu: Deprecate GtkTearoffMenuItem

testgtk: Remove tearoff menus from tests

gtk-demo: Don't use tearoff menus

Demoing bad things, sheesh.

gtkwindow: Clean up gtk_window_map()

Remove unneeded variable and delete trailing whitespace.

docs: Delete duplicate line in gdkwindow.c

handlebox: Deprecate

testgtk: Remove handlebox example

Rather than keeping it and enabling deprecated code in testgtk I decided
to delete the example. The warnings from testgtk about deprecated code
are too useful IMO.
Should we need it again, we can make it into its own test, or add a
testgtkdeprecated.

tests: Remove handleboxes from testmerge example

a11y: Don't include gtkx when not needed

a11y: Remove useless check

Updated Hebrew translation.

tests: Fix a bunch of set-but-unused warnings in the filtermodel test

The whole test doesn't seem to unref any memoery, somebody should review
it.

docs: Fix typo in treeview docs

gtktable: Fix gtk-doc notation

Replace some references to the GtkTable, use GtkGrid instead

notebook: don't forget to prepare the tab style context before drawing

The previous commit a9affcdd3f9c888c5e68dc76b0c5173d918bc725
accidentally removed the call to notebook_tab_prepare_style_context()

notebook: Remove unused variables

combobox: Remove unused define

GtkTreeViewColumn: initialize a variable

Shut up valgrind complaining about uninitialized values

==5330== Conditional jump or move depends on uninitialised value(s)
==5330==    at 0x624FF4C: gdk_window_move_resize_internal
(gdkwindow.c:6037)
==5330==    by 0x62502C3: gdk_window_move_resize (gdkwindow.c:6274)
==5330==    by 0x5E4FBB5: _gtk_tree_view_column_allocate
(gtktreeviewcolumn.c:2169)
==5330==    by 0x5E3846B: gtk_tree_view_size_allocate_columns
(gtktreeview.c:2631)
==5330==    by 0x5E43D76: gtk_tree_view_size_allocate
(gtktreeview.c:2686)
==5330==    by 0x7F03959: g_closure_invoke (gclosure.c:774)
==5330==    by 0x7F16446: signal_emit_unlocked_R (gsignal.c:3202)
==5330==    by 0x7F20256: g_signal_emit_valist (gsignal.c:3003)
==5330==    by 0x7F20421: g_signal_emit (gsignal.c:3060)
==5330==    by 0x5E6DFBE: gtk_widget_size_allocate (gtkwidget.c:4886)
==5330==    by 0x5D94ADD: gtk_scrolled_window_allocate_child
(gtkscrolledwindow.c:1506)
==5330==    by 0x5D95299: gtk_scrolled_window_size_allocate
(gtkscrolledwindow.c:1704)
==5330==    by 0x7F03959: g_closure_invoke (gclosure.c:774)
==5330==    by 0x7F16446: signal_emit_unlocked_R (gsignal.c:3202)
==5330==    by 0x7F20256: g_signal_emit_valist (gsignal.c:3003)
==5330==    by 0x7F20421: g_signal_emit (gsignal.c:3060)
==5330==    by 0x5E6DFBE: gtk_widget_size_allocate (gtkwidget.c:4886)
==5330==    by 0x5C62708: gtk_box_size_allocate (gtkbox.c:658)
==5330==    by 0x7F03959: g_closure_invoke (gclosure.c:774)
==5330==    by 0x7F16446: signal_emit_unlocked_R (gsignal.c:3202)
==5330==  Uninitialised value was created by a stack allocation
==5330==    at 0x5E4FA8D: _gtk_tree_view_column_allocate
(gtktreeviewcolumn.c:2139)

GdkWindow: fix conditional jump depends on uninitialized value error

Those if() blocks don't have any reason being there, as x and y are not
pointers. If the window is destroyed, just set the out values to zero
and return.

As seen in valgrind:

==3306== Conditional jump or move depends on uninitialised value(s)
==3306==    at 0x624C74F: gdk_window_get_root_coords (gdkwindow.c:6933)
==3306==    by 0x5E193C3: gtk_tooltip_show_tooltip (gtktooltip.c:1160)
==3306==    by 0x5E19C05: tooltip_popup_timeout (gtktooltip.c:1282)
==3306==    by 0x623B102: gdk_threads_dispatch (gdk.c:754)
==3306==    by 0x8592F3A: g_timeout_dispatch (gmain.c:3907)
==3306==    by 0x859174C: g_main_context_dispatch (gmain.c:2441)
==3306==    by 0x8591F47: g_main_context_iterate (gmain.c:3089)
==3306==    by 0x8592494: g_main_loop_run (gmain.c:3297)
==3306==    by 0x5D2E501: gtk_main (gtkmain.c:1362)
==3306==    by 0x5C5652F: gtk_application_run_mainloop
(gtkapplication.c:115)
==3306==    by 0x7C47C9D: g_application_run (gapplication.c:1323)
==3306==    by 0x447B5F: main (nautilus-main.c:102)
==3306==  Uninitialised value was created by a stack allocation
==3306==    at 0x624D48A: gdk_window_get_device_position
(gdkwindow.c:4952)

table: Deprecate GtkTable for GtkGrid

docs: Make infobar code example use GtkGrid instead of GtkTable

testtoolbar: Fix test to use GtkGrid

The conversion in commit 7724f14dfa7545cfb9f7a13cd0f2aa6a41018157 was
incomplete. Redo it.

testgtk: Convert panes example from GtkTable to GtkGrid

testgtk: Convert size groups example from GtkTable to GtkGrid

testgtk: Switch progress bar example from GtkTable to GtkGrid

testgtk: Clean up code

Remove unused hboxes.

quartz: don't filter away directly encoded 24-bit UCS characters

gdk_unicode_to_keyval(uc) returning (uc | 0x01000000) is not an
error return value but simply the way to encode 24-bit unicode
characters directly as keyvals.

testgtk: Move paned example from GtkTable to GtkGrid

testgtk: Move display example from GtkTable to GtkGrid

testgtk: Remove invalid property

testgtk: Convert focus example from GtkTable to GtkGrid

testgtk: Simplify code

Use a for loop for iterating.

testgtk: Switch scrolled windows example from GtkTable to GtkGrid

testgtk: Convert radio/check button examples from GtkTable to GtkGrid

print: Convert custom paper dialog from GtkTable to GtkGrid

testgtk: Convert buttons example from GtkTable to GtkGrid

testgtk: Beautify buttons example

testgtk: Convert big windows example from GtkTable to GtkGrid

testgtk: Upgrade alpha windows test from GtkTable to GtkGrid

Also fix some extra code.

tests: Convert testframe from GtkTable to GtkGrid

tests: Convert testgeometry from GtkTable to GtkGrid

tests: Convert testexpand from GtkTable to GtkGrid

tests: Convert testentryicons example from GtkTable to GtkGrid

tests: Convert testdnd from GtkTable to GtkGrid

tests: Convert testadjustsize example from GtkTable to box and grid

tests: Make prop-editor use a grid instead of a table

tests: Convert print-editor from GtkTable to GtkBox

tests: Convert testimage example from GtkTable to GtkGrid

tests: Convert testmerge from GtkTable to GtkGrid

toolbutton: don't force the icon widget to be a GtkMisc

There's really no reason the icon widget must be a GtkMisc. You might
want to use e.g. a GtkSpinner.

notebook: vborder/hborder deprecations are Since: 3.4, not 3.2

notebook: add top/bottom/left/right style classes to the tab region

When we use the style context to get information for the tab region,
also add a style class to indicate its position, so that the relevant
information is pulled off from the theme.

https://bugzilla.gnome.org/show_bug.cgi?id=659777

stylecontext: add style classes for top/bottom/right/left areas

This is useful to e.g. theme notebook tabs differently according to
their position directly from the CSS sheet.

GtkNotebook support in a separate commit.

https://bugzilla.gnome.org/show_bug.cgi?id=659777

notebook: unconditionally apply padding to the tab content

Not only when on left or top, otherwise the other position look
off-centered.

https://bugzilla.gnome.org/show_bug.cgi?id=659777

notebook: really deprecate tab_vborder and tab_hborder

The setter for this was deprecated in 2.x and removed in 3.0.
I don't see any reason why we should hardcode 2px for this value;
instead, deprecated the getter and make it always return zero, and stop
using the variables internally.

https://bugzilla.gnome.org/show_bug.cgi?id=659777

notebook: allow using different padding values for the active tab state

The code before was basically adding and removing the same padding value
in two different places during the allocation cycle.
Instead, what we want to do is to offset the inactive tab allocation by
the difference with the active tab padding, to ensure the tab content is
always drawn centered and in the right position.

https://bugzilla.gnome.org/show_bug.cgi?id=659777

notebook: use the current state to get the padding values

We want to enable the use of different padding values between active and
inactive tabs, so that the two are completely separated (but limited by
the active tab size).
This way themes can decide how bigger the active tab is drawn compared
to the normal one just specifying a different padding value from the
CSS, like this:

.notebook tab {
  padding: 2;
}

.notebook tab:active {
  padding: 4;
}

As a first step, fetch the padding values with the right state flags
from GtkStyleContext.

https://bugzilla.gnome.org/show_bug.cgi?id=659777

update zh_CN translation

mountoperation: fix the build

2d3db3421fd8067dd9c59a0727da239936187cc5 has a typo which broke the
build on master.

Updated POTFILES.skip

Bug 659406 - Abstract what triggers a context menu

Add gdk_event_triggers_context_menu(), using the new modifier
abstraction API. Remove _gtk_button_event_triggers_context_menu()
and port all callers.

GtkSettings: fix compile warnings on != X11

gtk: remove the private GTK_EXTEND/MODIFY_SELECTION_MOD_MASK

and use the new public modifier abstraction API instead.

gtk: remove the private GTK_NO_TEXT_INPUT_MOD_MASK

and use the new public modifier abstraction API.

Bug 659602 - Provide an abstraction for the platform's use of modifier keys

Add enum GdkModifierIntent which identifies use cases for modifier masks
and GdkKeyMap::get_modifier_mask(). Add a default implementation which returns
what is currently hardcoded all over GTK+, and an implementation in the
quartz backend. Also add gtk_widget_get_modifier_mask() which simplifies
things by doing widget->display->keymap->get_modifier_mask().

tests: Use a grid in testnotebookdnd instead of a table

tests: Use a grid in testorientable instead of a table

tests: Use a box instead of a table in testselection

toolbar: Fix widget path creation code

- Ensure arrow button is always included
- Invalidate list when direction changes